Skip to content
This repository was archived by the owner on Oct 15, 2021. It is now read-only.

A11y main #13

Merged
merged 5 commits into from
May 24, 2021
Merged

A11y main #13

merged 5 commits into from
May 24, 2021

Conversation

@storiesOfRen
Copy link
Collaborator

@storiesOfRen storiesOfRen commented May 24, 2021

  • Implementing @axe-core/react wrapper
  • Fixed A11y errors
  • Update to tests
    • Fix tests
    • Addition of semantics A11y test
  • Linting

@keonik keonik self-requested a review May 24, 2021 14:31
@keonik
Copy link
Collaborator

keonik commented May 24, 2021 •
edited
Loading

  • - Change Layout component to support SEO setting property and import via pages instead of _app.tsx

@keonik
Copy link
Collaborator

keonik commented May 24, 2021

@storiesOfRen

  • Update tools to include axe (lib/tools.ts)
  • Update docs to show added tooling (README.md)

@keonik keonik merged commit 0e79119 into MileTwo:main May 24, 2021
@keonik
Copy link
Collaborator

keonik commented Oct 14, 2021

@all-contributors please add @storiesOfRen for a11y

@allcontributors allcontributors bot mentioned this pull request Oct 14, 2021
Merged
@allcontributors
Copy link
Contributor

allcontributors bot commented Oct 14, 2021

@keonik

I've put up a pull request to add @storiesOfRen! 🎉

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@keonik keonik keonik approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@storiesOfRen @keonik